Skip to content

feat: calorie_result_title identifier for unique Calorie Result assertion - #4

Merged
thatswiftdev merged 1 commit into
mainfrom
feat/calorie-result-title-identifier
Oct 16, 2026
Merged

thatswiftdev merged 1 commit into
mainfrom
feat/calorie-result-title-identifier

Conversation

@thatswiftdev

Copy link
Copy Markdown
Contributor

What

One accessibility identifier on the Calorie Result header — .accessibilityIdentifier("calorie_result_title") (CalorieResultView.swift:67).
Modifier-chain addition only — no logic, layout, or copy changes.

Why

The header string (onboarding.result.title → "Estimasi target harian Anda adalah") is also rendered by GoalSetupView's live-target preview card (GoalSetupView.swift:192), and both screens stay mounted in the NavigationStack. A label-based E2E expect matched 3 elements — "bound not met — found 3, exactly 1" (Lionidas run_f8e763a4). A unique identifier gives the suite a locale-free, uniqueness-safe content assertion.

Test plan

  • CI: xcodegen + build (existing workflow)
  • No behavior change; optional smoke: onboarding → Calorie Result

…tion

The header string (onboarding.result.title) is also rendered by
GoalSetupView's live-target preview card, and both screens stay mounted
in the NavigationStack — a label-based E2E expect matched 3 elements
('bound not met — found 3, exactly 1', Lionidas run_f8e763a4). A unique
identifier gives the suite a locale-free, uniqueness-safe assertion.

Lionidas-In-Place: session sess_451c34f0-ceb, user-authorized
@thatswiftdev
thatswiftdev merged commit bb9369c into main Oct 16, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant